-
Notifications
You must be signed in to change notification settings - Fork 525
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
refactor: python test common class moved to a separate common assembly #1215
refactor: python test common class moved to a separate common assembly #1215
Conversation
@AsakusaRinne in this project nullable is enabled by default. I can change if needed. |
@AsakusaRinne I can convert it to fsproj if needed to force users use F# in tests. However, I think it doesn't make sense for a class. |
@AsakusaRinne how to get value of Ndarray if an Ndarray is a scalar? |
You can just use |
Error Message:
Test method TensorFlowNET.UnitTest.Gradient.GradientTest.testDependentYs threw exception:
System.NotImplementedException:
Stack Trace:
at Tensorflow.NumPy.np.allclose(NDArray a, NDArray b, Double rtol, Double atol, Boolean equal_nan) in /home/runner/work/TensorFlow.NET/TensorFlow.NET/src/TensorFlowNET.Core/Numpy/Numpy.cs:line 69
at TensorFlowNET.UnitTest.PythonTest.assertAllClose(Double value, NDArray array2, Double eps) in /home/runner/work/TensorFlow.NET/TensorFlow.NET/test/Tensorflow.UnitTest/PythonTest.cs:line 142
at TensorFlowNET.UnitTest.Gradient.GradientTest.testDependentYs() in /home/runner/work/TensorFlow.NET/TensorFlow.NET/test/TensorFlowNET.Graph.UnitTest/GradientTest/GradientTest.cs:line 600 |
8e3ea3a
to
7968dc3
Compare
Yes, one of the tests wasn't fully enabled before (in one of my previous PRs). Now the assertion works correctly and the error above is fixed. |
Thank you! I was confused by the fact that |
No description provided.